Skip to content

Added the definition of the SOCIAL_AUTH_LOGIN_REDIRECT_URL variable#13428

Merged
Maffooch merged 1 commit intoDefectDojo:devfrom
rseleven:master
Oct 17, 2025
Merged

Added the definition of the SOCIAL_AUTH_LOGIN_REDIRECT_URL variable#13428
Maffooch merged 1 commit intoDefectDojo:devfrom
rseleven:master

Conversation

@rseleven
Copy link
Copy Markdown
Contributor

When the DD_LOGIN_REDIRECT_URL variable was set, the redirect URL was not applied when using OIDC authentication.

Check in the browser in incognito mode or with a cleared cache.

@github-actions github-actions Bot added the settings_changes Needs changes to settings.py based on changes in settings.dist.py included in this PR label Oct 16, 2025
@mtesauro
Copy link
Copy Markdown
Contributor

@rseleven Please make your PR against the DEV branch - we don't accept PRs against main/master

See https://github.com/DefectDojo/django-DefectDojo/blob/master/readme-docs/CONTRIBUTING.md#submitting-pull-requests for details

@rseleven rseleven changed the base branch from master to dev October 16, 2025 19:05
@rseleven
Copy link
Copy Markdown
Contributor Author

corrected

Copy link
Copy Markdown
Contributor

@mtesauro mtesauro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@valentijnscholten valentijnscholten added this to the 2.52.0 milestone Oct 17, 2025
Copy link
Copy Markdown
Member

@valentijnscholten valentijnscholten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rseleven thanks for the PR

@Maffooch Maffooch merged commit dec5a63 into DefectDojo:dev Oct 17, 2025
147 checks passed
Maffooch pushed a commit to valentijnscholten/django-DefectDojo that referenced this pull request Feb 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

settings_changes Needs changes to settings.py based on changes in settings.dist.py included in this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants